Skip to content

plugins/hunk: init #3293

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 12, 2025
Merged

Conversation

jalil-salame
Copy link
Contributor

Add support for hunk.nvim, a plugin for splitting diffs.

Also add myself as a maintainer for it.

Copy link
Member

@GaetanLepage GaetanLepage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Here is some initial feedback.

Comment on lines 14 to 18
extraOptions = {
fileTypeIcons = lib.mkEnableOption ''
Enable file type icons in the file tree.
'';
};
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have missed something or this option is only used in the warning?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, only in the warning, I tried setting web-devicons to true there, but it threw a warning.

@jalil-salame jalil-salame force-pushed the push-yxsyxkonwotv branch 2 times, most recently from 3de2637 to 714e66e Compare May 7, 2025 07:29
@jalil-salame jalil-salame requested a review from GaetanLepage May 7, 2025 07:30
Comment on lines 14 to 18
extraOptions = {
fileTypeIcons = lib.mkEnableOption ''
warning if file type icons are not available.
'';
};
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having an enable option for a warning feels like an anti-pattern

Suggested change
extraOptions = {
fileTypeIcons = lib.mkEnableOption ''
warning if file type icons are not available.
'';
};

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the option!

Adding myself as a maintainer
Add support for [hunk.nvim][1], a plugin for splitting diffs.

Also add myself as a maintainer for it.

[1]: https://github.com/julienvincent/hunk.nvim
@MattSturgeon
Copy link
Member

Thanks!

@MattSturgeon

This comment was marked as resolved.

This comment was marked as resolved.

Copy link
Contributor

mergify bot commented May 12, 2025

This pull request, with head sha 69266437beaac970568d12a46766706c72f361b3, has been successfully merged with fast-forward by Mergify.

This pull request will be automatically closed by GitHub.

As soon as GitHub detects that the sha 69266437beaac970568d12a46766706c72f361b3 is part of the main branch, it will mark this pull request as merged.

It is possible for this pull request to remain open if this detection does not happen, this usually happens when a force-push is done on this branch push-yxsyxkonwotv, this means GitHub will fail to detect the merge.

@mergify mergify bot merged commit 6926643 into nix-community:main May 12, 2025
4 checks passed
@mergify mergify bot temporarily deployed to github-pages May 12, 2025 07:05 Inactive
@jalil-salame jalil-salame deleted the push-yxsyxkonwotv branch May 12, 2025 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants